Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add GPU tracking to Jobs realm #1270

Merged
merged 4 commits into from
Mar 26, 2020

Conversation

jtpalmer
Copy link
Contributor

Description

Adds GPU statistics and a group by to the jobs realm. Only includes support for Slurm at this time.

Replaces #1235

Motivation and Context

https://app.asana.com/0/15230011293693/1145503330514643

Tests performed

Added additional tests and regression test data.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project as found in the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@jtpalmer jtpalmer added Category:ETL Extract Transform Load new feature New functionality Category:General General labels Mar 19, 2020
@jtpalmer jtpalmer added this to the 9.0.0 milestone Mar 19, 2020
@jtpalmer jtpalmer mentioned this pull request Mar 19, 2020
6 tasks
$console = Console::factory();
$console->displayMessage(<<<"EOT"
This version of Open XDMoD has support for GPU metrics in the jobs realm. If
you have shredded Slurm job records in the past it is possible to extract the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What version of XDMoD was the reqgres data added? Or has the capability been there effectively forever?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was added in 6.5 (released 2017-01-10).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps just a note in the markdown docs about this. I doubt many people have pre 6.5 data in their database.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a note to the upgrade documentation.

@jtpalmer jtpalmer merged commit 73301c6 into ubccr:xdmod9.0 Mar 26, 2020
@jtpalmer jtpalmer deleted the job-realm-gpu-refactored branch March 26, 2020 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:ETL Extract Transform Load Category:General General new feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants